Hide Pro by Alek99 · Pull Request #1688 · reflex-dev/reflex-web

Alek99 · 2025-11-12T00:20:10Z

No description provided.

greptile-apps · 2025-11-12T00:23:58Z

Greptile Overview

Greptile Summary

Temporarily hides the Pro tier from the pricing page by commenting out the Pro plan card and Pro column in comparison tables, adjusting layouts from 3-column to 2-column grid (Hobby + Enterprise only).

Key Changes:

Commented out Pro tier pricing card in plan_cards.py
Updated grid layout from xl:grid-cols-3 to xl:grid-cols-2 with centered alignment
Commented out Pro column in sticky pricing header and all feature comparison tables
Changed table grid from grid-cols-4 to grid-cols-3

Critical Issues:

Violates repository policy requiring removal of commented-out code before merging (rule d49e2a0e)
Unused imports: REFLEX_CLOUD_URL, PRO_TIERS_TABLE
Unused code: ProTierState class, pro_tiers_select() function, monthly_yearly_toggle() function, monthly_yearly_toggle_cs client state
Feature tuple arrays contain inline comments for middle values creating confusing data structures

Required Actions Before Merge:

Remove all commented-out code blocks
Delete unused state class, functions, and imports
Clean up feature tuples to only contain 2 values (not 3 with middle commented)

Confidence Score: 0/5

This PR cannot be merged in its current state due to critical policy violations
Score of 0 reflects multiple critical violations of repository policy (d49e2a0e): extensive commented-out code throughout both files, unused imports, and entire unused classes/functions. The PR violates the explicit rule requiring removal of commented-out code before merging. While the actual functional changes (hiding Pro tier, adjusting layouts) are straightforward and low-risk, the code quality issues make this unsuitable for merge without cleanup.
Both pcweb/pages/pricing/plan_cards.py and pcweb/pages/pricing/table.py require immediate attention to remove all commented code and unused definitions before this PR can be merged

Important Files Changed

File Analysis

Filename	Score	Overview
pcweb/pages/pricing/plan_cards.py	1/5	Commented out Pro tier card and adjusted grid layout; contains significant unused code (ProTierState, helper functions, imports) that must be removed per repository rules
pcweb/pages/pricing/table.py	1/5	Commented out Pro tier column in pricing tables and sticky header; extensive commented-out code in feature tuples that must be removed per repository rules

Sequence Diagram

sequenceDiagram
    participant User
    participant PricingPage
    participant PlanCards
    participant TiersTable
    
    User->>PricingPage: Visit pricing page
    PricingPage->>PlanCards: Render plan_cards()
    PlanCards->>PlanCards: Show Hobby card (Free tier)
    PlanCards->>PlanCards: Skip Pro card (commented out)
    PlanCards->>PlanCards: Show Enterprise card (popular)
    PlanCards-->>PricingPage: Display 2-column grid layout
    
    PricingPage->>TiersTable: Render tiers_tables()
    TiersTable->>TiersTable: Show sticky header (Hobby + Enterprise)
    TiersTable->>TiersTable: Skip Pro column (commented out)
    TiersTable->>TiersTable: Render feature tables (2 columns)
    TiersTable-->>PricingPage: Display comparison tables
    
    User->>PlanCards: Click "Start building for free"
    PlanCards-->>User: Redirect to REFLEX_BUILD_URL
    
    User->>PlanCards: Click "Contact sales"
    PlanCards-->>User: Show demo form dialog

greptile-apps

Additional Comments (6)

pcweb/pages/pricing/plan_cards.py, line 13-18 (link)

logic: YEARLY_MONTHS_FREE, YEARLY_DISCOUNT_MULTIPLIER, and monthly_yearly_toggle_cs are defined but never used - remove them

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)
pcweb/pages/pricing/plan_cards.py, line 26-45 (link)

logic: ProTierState class is defined but never used - remove it

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)
pcweb/pages/pricing/plan_cards.py, line 160-173 (link)

logic: pro_tiers_select() function is defined but never used - remove it

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)
pcweb/pages/pricing/plan_cards.py, line 284-304 (link)

logic: monthly_yearly_toggle() function is defined but never used - remove it

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)
pcweb/pages/pricing/plan_cards.py, line 310 (link)

logic: Remove commented-out code before merging

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)
pcweb/pages/pricing/table.py, line 131-150 (link)

logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

_{2 files reviewed, 15 comments}

_{Edit Code Review Agent Settings | Greptile}

greptile-apps · 2025-11-12T00:23:50Z

pcweb/pages/pricing/plan_cards.py

 from pcweb.components.hosting_banner import HostingBannerState
-from pcweb.components.number_flow import number_flow
 from pcweb.constants import PRO_TIERS_TABLE, REFLEX_BUILD_URL, REFLEX_CLOUD_URL



logic: REFLEX_CLOUD_URL is imported but never used - remove it

Suggested change

from pcweb.constants import PRO_TIERS_TABLE, REFLEX_BUILD_URL

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/plan_cards.py Line: 11:11 Comment: **logic:** `REFLEX_CLOUD_URL` is imported but never used - remove it ```suggestion from pcweb.constants import PRO_TIERS_TABLE, REFLEX_BUILD_URL ``` **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:52Z

pcweb/pages/pricing/plan_cards.py

+        #     "Pro",
+        #     number_flow(
+        #         value=rx.cond(
+        #             monthly_yearly_toggle_cs.value == "monthly",
+        #             ProTierState.selected_tier["price"],
+        #             round(
+        #                 ProTierState.selected_tier["price"]
+        #                 * YEARLY_DISCOUNT_MULTIPLIER,
+        #                 1,
+        #             ),
+        #         ),
+        #         trend="0",
+        #         prefix="$",
+        #         suffix=" /monthly",
+        #         class_name="text-3xl text-secondary-12 font-semibold py-4",
+        #     ),
+        #     "Build, deploy and scale your apps.",
+        #     [
+        #         Feature("TokenCircleIcon", "", pro_tiers_select()),
+        #         Feature("SquareLock02Icon", "Private Projects"),
+        #         Feature("CursorInWindowIcon", "Full-Fledged Browser IDE"),
+        #         Feature("PlugSocketIcon", "Integrations"),
+        #         Feature("GithubIcon", "Connect to Github"),
+        #         Feature("Globe02Icon", "Custom Domains"),
+        #         Feature("CpuIcon", "Up to 8 GB RAM / 4 vCPU per deployed app"),
+        #     ],
+        #     ui.button(
+        #         "Start with Pro plan",
+        #         variant="secondary",
+        #         size="lg",
+        #         class_name="w-full font-semibold",
+        #         on_click=ProTierState.redirect_to_billing(
+        #             monthly_yearly_toggle_cs.value == "yearly"
+        #         ),
+        #     ),
+        # ),
        popular_card(


logic: Remove commented-out code before merging

Suggested change

# "Pro",

# number_flow(

# value=rx.cond(

# monthly_yearly_toggle_cs.value == "monthly",

# ProTierState.selected_tier["price"],

# round(

# ProTierState.selected_tier["price"]

# * YEARLY_DISCOUNT_MULTIPLIER,

# 1,

# ),

# ),

# trend="0",

# prefix="$",

# suffix=" /monthly",

# class_name="text-3xl text-secondary-12 font-semibold py-4",

# ),

# "Build, deploy and scale your apps.",

# [

# Feature("TokenCircleIcon", "", pro_tiers_select()),

# Feature("SquareLock02Icon", "Private Projects"),

# Feature("CursorInWindowIcon", "Full-Fledged Browser IDE"),

# Feature("PlugSocketIcon", "Integrations"),

# Feature("GithubIcon", "Connect to Github"),

# Feature("Globe02Icon", "Custom Domains"),

# Feature("CpuIcon", "Up to 8 GB RAM / 4 vCPU per deployed app"),

# ],

# ui.button(

# "Start with Pro plan",

# variant="secondary",

# size="lg",

# class_name="w-full font-semibold",

# on_click=ProTierState.redirect_to_billing(

# monthly_yearly_toggle_cs.value == "yearly"

# ),

# ),

# ),

popular_card(

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/plan_cards.py Line: 202:238 Comment: **logic:** Remove commented-out code before merging ```suggestion ``` **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:53Z

pcweb/pages/pricing/table.py


 from pcweb.components.hosting_banner import HostingBannerState
-from pcweb.constants import REFLEX_BUILD_URL, REFLEX_CLOUD_URL
+from pcweb.constants import REFLEX_BUILD_URL  # , REFLEX_CLOUD_URL


logic: REFLEX_CLOUD_URL is commented out but never used - remove the commented import

Suggested change

from pcweb.constants import REFLEX_BUILD_URL # , REFLEX_CLOUD_URL

from pcweb.constants import REFLEX_BUILD_URL

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 6:6 Comment: **logic:** `REFLEX_CLOUD_URL` is commented out but never used - remove the commented import ```suggestion from pcweb.constants import REFLEX_BUILD_URL ``` **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:54Z

pcweb/pages/pricing/table.py

 CLOUD_HOSTING_FEATURES = [
-    ("Max # Apps", "1", "5", "10"),
-    ("Max Machine Size", "1cpu, 1gb", "2cpu, 4gb shared", "Beyond 2cpu, 4gb"),
-    ("Dedicated Machines", False, False, True),
-    ("Custom Domains", "0", "5", "Unlimited"),
-    ("App Metrics", True, True, True),
-    ("Log Retention", "1 hour", "7 day", "90-Day Log History"),
-    ("Multiple Regions", False, True, True),
-    ('"Built with Reflex" Attribution', True, False, False),
-    ("One-click rollbacks", True, True, True),
+    (
+        "Max # Apps",
+        "1",
+        # "5",
+        "10",
+    ),
+    (
+        "Max Machine Size",
+        "1cpu, 1gb",
+        # "2cpu, 4gb shared",
+        "Beyond 2cpu, 4gb",
+    ),
+    (
+        "Dedicated Machines",
+        False,
+        # False,
+        True,
+    ),
+    (
+        "Custom Domains",
+        "0",
+        # "5",
+        "Unlimited",
+    ),
+    (
+        "App Metrics",
+        True,
+        # True,
+        True,
+    ),
+    (
+        "Log Retention",
+        "1 hour",
+        # "7 day",
+        "90-Day Log History",
+    ),
+    (
+        "Multiple Regions",
+        False,
+        # True,
+        True,
+    ),
+    (
+        '"Built with Reflex" Attribution',
+        True,
+        # False,
+        False,
+    ),
+    (
+        "One-click rollbacks",
+        True,
+        # True,
+        True,
+    ),
 ]


logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 8:63 Comment: **logic:** Remove commented-out middle column values (Pro tier) from feature tuples The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:54Z

pcweb/pages/pricing/table.py

 SECURITY_FEATURES = [
-    ("SSO/SAML", False, True, True),
-    ("Role-based access control", False, False, True),
-    ("On Premise Deployments", False, False, True),
-    ("Audit Logs", False, False, True),
-    ("HTTP/SSL", True, True, True),
-    ("Web App Firewall", True, True, True),
-    ("SOC 2 compliance", False, False, "On prem, custom"),
-    ("HIPAA BAA", False, False, "On prem, custom"),
+    (
+        "SSO/SAML",
+        False,
+        # True,
+        True,
+    ),
+    (
+        "Role-based access control",
+        False,
+        # False,
+        True,
+    ),
+    (
+        "On Premise Deployments",
+        False,
+        # False,
+        True,
+    ),
+    (
+        "Audit Logs",
+        False,
+        # False,
+        True,
+    ),
+    (
+        "HTTP/SSL",
+        True,
+        # True,
+        True,
+    ),
+    (
+        "Web App Firewall",
+        True,
+        # True,
+        True,
+    ),
+    (
+        "SOC 2 compliance",
+        False,
+        # False,
+        "On prem, custom",
+    ),
+    (
+        "HIPAA BAA",
+        False,
+        # False,
+        "On prem, custom",
+    ),
 ]


logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 65:114 Comment: **logic:** Remove commented-out middle column values (Pro tier) from feature tuples The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:55Z

pcweb/pages/pricing/table.py

 SUPPORT_FEATURES = [
    (
        "Customer Success",
        "Discord/Github Community",
-        "Discord/Github Community",
+        # "Discord/Github Community",
        "Dedicated Support Channel",
    ),
    (
        "Onboarding",
        "Documentation",
-        "Documentation",
+        # "Documentation",
        "Get a forward deployed engineer to help you get started",
    ),
 ]


logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 116:129 Comment: **logic:** Remove commented-out middle column values (Pro tier) from feature tuples The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:56Z

pcweb/pages/pricing/table.py

 REFLEX_BUILD_FUNCTIONALITY = [
    (
        "Privacy",
        "Public Projects",
-        "Private Projects",
+        # "Private Projects",
        "Private Projects /Group based controls",
    ),
    (
        "Design",
        "Custom Designs/Theming",
+        # "Custom Designs/Theming",
        "Custom Designs/Theming",
-        "Custom Designs/Theming",
    ),
-    ("Data", False, False, "Opt out of data training"),
-    ("Collaborators", "Single", "Single", "Multiple Collaborators/Editors"),
-    ("Integration", "Basic 5", "Pro 100+", "Enterprise Integrations"),
-    ("Download App Code", False, True, True),
+    (
+        "Data",
+        False,
+        # False,
+        "Opt out of data training",
+    ),
+    (
+        "Collaborators",
+        "Single",
+        # "Single",
+        "Multiple Collaborators/Editors",
+    ),
+    (
+        "Integration",
+        "Basic 5",
+        # "Pro 100+",
+        "Enterprise Integrations",
+    ),
+    (
+        "Download App Code",
+        False,
+        # True,
+        True,
+    ),
 ]


logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 152:189 Comment: **logic:** Remove commented-out middle column values (Pro tier) from feature tuples The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:56Z

pcweb/pages/pricing/table.py

 REFLEX_BUILD_DEPLOYMENT = [
    (
        "Github",
        "Public Repo Sync",
-        "Private Repo Sync",
+        # "Private Repo Sync",
        "Enterprise Repo Sync Github, Gitlab, and Bitbucket.",
    ),
    (
        "One Click Deploy",
        "Reflex Cloud",
-        "Reflex Cloud",
+        # "Reflex Cloud",
        "Databricks, AWS, Azure, GCP, Other",
    ),
-    ("SSH access", False, True, True),
+    (
+        "SSH access",
+        False,
+        # True,
+        True,
+    ),
 ]


logic: Remove commented-out middle column values (Pro tier) from feature tuples

The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 191:210 Comment: **logic:** Remove commented-out middle column values (Pro tier) from feature tuples The tuples should only contain 2 values (Hobby and Enterprise), not 3 with the middle one commented out **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

greptile-apps · 2025-11-12T00:23:57Z

pcweb/pages/pricing/table.py

+            # header_item(
+            #     "Pro",
+            #     ui.link(
+            #         render_=ui.button(
+            #             "Upgrade now",
+            #             variant="secondary",
+            #             class_name="font-semibold w-full",
+            #         ),
+            #         to=f"{REFLEX_CLOUD_URL.rstrip('/')}/?redirect_url={REFLEX_CLOUD_URL.rstrip('/')}/billing/",
+            #         target="_blank",
+            #     ),
+            # ),


logic: Remove commented-out Pro tier header

Suggested change

# header_item(

# "Pro",

# ui.link(

# render_=ui.button(

# "Upgrade now",

# variant="secondary",

# class_name="font-semibold w-full",

# ),

# to=f"{REFLEX_CLOUD_URL.rstrip('/')}/?redirect_url={REFLEX_CLOUD_URL.rstrip('/')}/billing/",

# target="_blank",

# ),

# ),

Context Used: Rule from dashboard - Remove commented-out code before merging PRs. (source)

Prompt To Fix With AI

This is a comment left during a code review. Path: pcweb/pages/pricing/table.py Line: 338:349 Comment: **logic:** Remove commented-out Pro tier header ```suggestion ``` **Context Used:** Rule from `dashboard` - Remove commented-out code before merging PRs. ([source](https://app.greptile.com/review/custom-context?memory=d49e2a0e-27a4-4cd6-b764-58c8a6fc4032)) How can I resolve this? If you propose a fix, please make it concise.

Temporarily hide Pro tier

66c0a07

Alek99 changed the title ~~Temporarily hide Pro tier~~ Hide Pro Nov 12, 2025

Fix lint after hiding Pro tier

1d7f6c4

greptile-apps bot reviewed Nov 12, 2025

View reviewed changes

picklelo approved these changes Nov 12, 2025

View reviewed changes

Alek99 merged commit 06d1b4b into main Nov 12, 2025
10 checks passed

Alek99 deleted the remove-pro-tier branch November 12, 2025 00:38

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Hide Pro#1688

Hide Pro#1688
Alek99 merged 2 commits intomainfrom
remove-pro-tier

Alek99 commented Nov 12, 2025

Uh oh!

greptile-apps bot commented Nov 12, 2025

Uh oh!

greptile-apps bot left a comment •

edited

Loading

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

greptile-apps bot Nov 12, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants


	from pcweb.constants import PRO_TIERS_TABLE, REFLEX_BUILD_URL

	from pcweb.constants import REFLEX_BUILD_URL # , REFLEX_CLOUD_URL
	from pcweb.constants import REFLEX_BUILD_URL

Conversation

Alek99 commented Nov 12, 2025

Uh oh!

greptile-apps bot commented Nov 12, 2025

Greptile Overview

Greptile Summary

Confidence Score: 0/5

Important Files Changed

Sequence Diagram

Uh oh!

greptile-apps bot left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Additional Comments (6)

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

greptile-apps bot Nov 12, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

greptile-apps bot left a comment •

edited

Loading